Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nix-profile.fish: Add local state bin to $PATH #12423

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ilya-bobyr
Copy link
Contributor

Motivation

It seems reasonable to add both $HOME/.profile/bin and @localstatedir@/nix/profiles/default/bin to $PATH for both user local and daemon based nix execution.
Nix daemon execution mode does not affect these paths.

Context

Further reduce differences between nix-profile.fish and nix-profile-daemon.fish.
A higher level list of all the differences is here.


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@ilya-bobyr ilya-bobyr requested a review from edolstra as a code owner February 5, 2025 00:13
It seems reasonable to add both `$HOME/.profile/bin` and
`@localstatedir@/nix/profiles/default/bin` to `$PATH` for both user
local and daemon based nix execution.  Nix daemon execution mode does
not affect these path.
@ilya-bobyr ilya-bobyr force-pushed the fish-profile-local-state-bin-in-PATH branch from 35b80d5 to c09b6da Compare February 5, 2025 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant